Conversation
|
I just saw that, weird, I'll try to fix it |
remove unwanted files
|
it's fixed now, removed those files |
| func configureStyling() { | ||
|
|
||
| window?.tintColor = UIColor.trySwiftNavigationBarColor() | ||
| window?.tintColor = UIColor.trySwiftNavigationBarColor |
There was a problem hiding this comment.
You can omit the UIColor prefixes as these are inferred.
| return cell | ||
| } | ||
|
|
||
|
|
| acknowledgementesViewController.headerText = "We ❤️ Open Source Software".localized() | ||
|
|
||
| performSegue(withIdentifier: moreDetailSegue, sender: acknowledgementesViewController) | ||
| func navigateTo<T: UIViewController>(id: String = "moreDetailSegue", _ controller: () -> T){ |
There was a problem hiding this comment.
🤔, I don't really like this signature. Especially when used without spacing for the {}'s. What about func navigate<T: UIViewController>(to controller: () -> T, id: String = "moreDetailSegue")? Then use it with the parameter:
navigate(to: { acknowledgement })Thoughts, @NatashaTheRobot?
There was a problem hiding this comment.
Now that I think of it... is there any reason why this couldn't just accept a UIViewController instead of a closure?
There was a problem hiding this comment.
Sounds much simpler that way!
There was a problem hiding this comment.
Looks better indeed, I've replaced it
trySwift/UIImage+Downloader.swift
Outdated
| @@ -0,0 +1,27 @@ | |||
| // | |||
There was a problem hiding this comment.
It was an example to separate 3rd library from code so it's to be replaced easily anytime, I'll remove it
trySwift/UITableViewExtension.swift
Outdated
|
|
||
| let nib = UINib(nibName: T.nibName, bundle: nil) | ||
| register(nib, forCellReuseIdentifier: T.reuseIdentifier) | ||
|
|
3 participants
A few updates for code syntax:
moving uicolor functions extension to vars for more swifty way
rewriting navigate to function in settings to be generic with less lines
this was done as a part of our community activity (reading code club) in #swiftCairo
thanks